Skip to content

Conversation

@glessard
Copy link
Contributor

@glessard glessard commented Mar 21, 2025

Many tests are conditionally excluded from compilation on WASI, and this is error-prone. Tests should instead be explicitly skipped in each case. By doing this on only one file, several wrongly-excluded test cases are restored to the WASI test suite.

Explicitly skip `expectCrashLater()` tests for WASI, instead of conditionally excluding them from compilation.

In doing so, restored some tests that were wrongly excluded from the WASI tests suite.
@glessard
Copy link
Contributor Author

@swift-ci please test Linux platform

@glessard
Copy link
Contributor Author

@swift-ci please smoke test Windows platform

@glessard
Copy link
Contributor Author

@swift-ci please test Linux platform

@glessard glessard marked this pull request as ready for review March 22, 2025 00:17
@glessard glessard requested a review from a team as a code owner March 22, 2025 00:17
@glessard
Copy link
Contributor Author

@swift-ci please smoke test macOS platform

@glessard glessard enabled auto-merge March 22, 2025 02:43
@glessard
Copy link
Contributor Author

@swift-ci please smoke test macOS platform

@glessard glessard merged commit ebbec50 into swiftlang:main Mar 23, 2025
4 checks passed
@glessard glessard deleted the stdlibUnittest-wasi-skip branch March 25, 2025 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants